-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add interval option #34
Conversation
4d2fdc5
to
10bd8e1
Compare
Hi @kyleyannelli |
@narugit I like the Edit: If you'd like I will have more time over the weekend to make a test that can just try every combination of interval by steps of 10 and every retry by steps of 100 to see which combo yields the best results. I'm quite confident with my current values, but I can't help but be a bit curious about the possibility of a better combo. |
@kyleyannelli As a separate idea from the interval option, how about adding an option (-f, --fail-soft) that saves the most recent valid sensor value in |
That sounds good to me! I'm currently running smctemp in a loop through a system service. Saving the last non zero value in a text file so I can just run |
10bd8e1
to
17cb9ae
Compare
README.md
Outdated
For M2 Macs, using the `-n`, `-i`, and `-f` options can help obtain more stable sensor values. | ||
Try tuning these options to get better results. | ||
|
||
The recommended settings are `-i25 -n40 -f` (See also https://github.com/narugit/smctemp/issues/32#issuecomment-2287304793). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @kyleyannelli
I'd like to add recommended settings for M2 user, so could you provide your recommended settings with -f
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @narugit
After messing around a bit I'm going to recommend -i25 -n180 -f
. The lower retries just don't hit even when the CPU is under load. I think 25ms and 180 maximum retries work well enough to get a result while the max wait time is under 5 seconds.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kyleyannelli
Thank you for checking! I've updated the recommended settings.
17cb9ae
to
38a671c
Compare
About
-i
and-n
options to obtain more stable and accurate temperature readings.